Skip to content

Tox cleanup - #119

Merged
mahmoud merged 3 commits into
masterfrom
tox-cleanup
Apr 2, 2020
Merged

mahmoud merged 3 commits into
masterfrom
tox-cleanup

Conversation

@wsanchez

@wsanchez wsanchez commented Apr 2, 2020

Copy link
Copy Markdown
Contributor

This cleans up a few build files, and should reduce some extra noise in #106.

Comment thread MANIFEST.in Resolved
Comment thread tox.ini Resolved
Comment thread tox.ini Resolved
Comment thread tox.ini Resolved
Comment thread tox.ini Resolved
@codecov-io

codecov-io commented Apr 2, 2020 •

Copy link
Copy Markdown

Codecov Report

Merging #119 into master will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #119   +/-   ##
=======================================
  Coverage   98.55%   98.55%           
=======================================
  Files          10       10           
  Lines        1595     1595           
  Branches      187      187           
=======================================
  Hits         1572     1572           
  Misses         12       12           
  Partials       11       11           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 900c9de...a08690c. Read the comment docs.

Comment thread tox.ini Resolved

@mahmoud mahmoud left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lol did you... review your own PR? I think we're all getting a bit stir crazy, but keep channeling it productively! LGTM, good cleanup!

@mahmoud
mahmoud merged commit 4ba4e82 into master Apr 2, 2020
@wsanchez

wsanchez commented Apr 2, 2020

Copy link
Copy Markdown
Contributor Author

I wasn't reviewing it… :-)
…just adding notes to make it easier to review.

@wsanchez
wsanchez deleted the tox-cleanup branch April 2, 2020 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants